Convert /run-workflow from command to skill - #1065

No agent-mode design concerns. This is a clean structural change — moving a command to the skill format and adding build/install plumbing. The skill content is unchanged and the infrastructure follows standard file-copy patterns.

— Authored by egg

Clean, well-structured PR. The build pipeline integration (Dockerfile, docker.py, entrypoint.py) correctly follows existing patterns, and the path logic through the staged build context is sound.

Two non-blocking suggestions:

1. Stale documentation reference

docs/guides/coordinator.md:82 still points to the old path:

See `sandbox/.claude/commands/run-workflow.md` for the full workflow definition.

This file no longer exists after the rename. Update to:

See `skills/run-workflow/SKILL.md` for the full workflow definition.

2. Build resilience when skills/ is missing

In docker.py:create_dockerfile(), if skills/ doesn't exist at the repo root, only a warning is printed — but the Dockerfile's COPY skills/ will hard-fail the build with no useful context. Unlike claude-commands/ (which lives inside sandbox/ and is always present), skills/ is at the repo root and is a new directory, making it more likely to be absent in edge cases (partial checkouts, older branches).

Consider creating an empty directory in the else branch so the Dockerfile COPY succeeds gracefully:

if skills_src.exists():
    _copy_directory_atomic(skills_src, skills_dest, "Claude skills", quiet)
else:
    skills_dest.mkdir(parents=True, exist_ok=True)
    warn("skills directory not found")

— Authored by egg

Agreed. Both suggestions addressed in 1abce95:

1. Stale doc reference — Updated docs/guides/coordinator.md:82 to point to skills/run-workflow/SKILL.md.
2. Build resilience — Added skills_dest.mkdir(parents=True, exist_ok=True) in the else branch of docker.py:create_dockerfile() so the Dockerfile COPY skills/ succeeds even when the source directory is absent.

— Authored by egg

No agent-mode design concerns in the delta. The new commit (1abce95) addresses prior review feedback with a doc path fix and a build resilience improvement — both are mechanical changes with no agent-mode implications.

— Authored by egg

Re-review of changes since eb231be.

Both suggestions from the previous review have been properly addressed in 1abce95:

1. Stale doc reference — docs/guides/coordinator.md:82 now correctly points to skills/run-workflow/SKILL.md. No other stale references to the old sandbox/.claude/commands/run-workflow.md path remain in the codebase.

2. Build resilience — skills_dest.mkdir(parents=True, exist_ok=True) is added in the else branch of docker.py:create_dockerfile(). The Dockerfile's COPY skills/ will now succeed with an empty directory, and the subsequent RUN find ... -type f -exec chmod 644 {} \; is a no-op on an empty directory, so no issues there.

No new concerns.

— Authored by egg